-
Notifications
You must be signed in to change notification settings - Fork 612
Don't use deprecated asmjit templates #4481
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for pytorch-fbgemm-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@q10 gtest also failed because of deprecated functions, now it is fixed. |
Signed-off-by: cyy <cyyever@outlook.com>
Signed-off-by: cyy <cyyever@outlook.com>
Signed-off-by: cyy <cyyever@outlook.com>
@q10 I disable another warning on MacOS CLAPACK. Since FBGEMM doesn't build on Apple Silicon (for no AVX512), I temporarily disable it. Will recheck the decision later when I can build it on M2. |
@q10 Let's restore the option but keep the source code changes because I don't have Intel Macbook at hand.. |
@cyyever It looks like the changes break the macOS CI builds. |
d60c852
to
943be88
Compare
Signed-off-by: cyy <cyyever@outlook.com>
@q10 More reverts |
Summary: X-link: facebookresearch/FBGEMM#1550 Clean up these deprecation warnings before it is possible to build newer asmjit. Pull Request resolved: pytorch#4481 Reviewed By: cthi Differential Revision: D78252575 Pulled By: q10
Clean up these deprecation warnings before it is possible to build newer asmjit.